Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flags Updates #1096

Merged
merged 10 commits into from
Apr 14, 2023
Merged

Flags Updates #1096

merged 10 commits into from
Apr 14, 2023

Conversation

sadmuphin
Copy link
Contributor

@sadmuphin sadmuphin commented Apr 13, 2023

Background

The Auto-GPT script should be more user friendly and more "scriptable" so to speak.

Changes

Added the following flags:

  • --skip-reprompt (-y) - Skips the Auto-GPT re-prompt message if the ai_setting.yaml file exists. Helpful for testing.
  • --ai-settings (-C) - Allows the user to pass through a separate AI Settings file. Useful to quickly switch between AI Configurations, etc. (Also reads value from the AI_SETTINGS_FILE environment variable, if user wants to specify non-default file without using the flag every time). Will also skip the re-prompt.
  • Added a shorthand for --continuous: -c
  • Fixed the error logging with --use-memory flag. Changed print_to_console() -> logger.typewriter_log()

Note: the program will validate if the user-provided AI Settings files are correct. It does this by checking for the files existence and ensures the YAML can be read and loaded properly.

Test Plan

Tested by running each flag individually and with each other, everything seems to work correctly. If people could also help test on other systems (Linux, MacOS, etc.) that would be great!

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes to the best of my ability
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes

@nponeccop nponeccop mentioned this pull request Apr 13, 2023
1 task
nponeccop
nponeccop previously approved these changes Apr 13, 2023
@richbeales
Copy link
Contributor

I think .env.template and README.md should be altered to surface your changes to the user.

@nponeccop
Copy link
Contributor

@cryptidv There are conflicts now

@sadmuphin sadmuphin requested a review from nponeccop April 13, 2023 22:21
@sadmuphin
Copy link
Contributor Author

I think .env.template and README.md should be altered to surface your changes to the user.

Done.

@nponeccop Everything should be resolved now

@sadmuphin sadmuphin marked this pull request as draft April 13, 2023 22:38
@sadmuphin sadmuphin marked this pull request as ready for review April 13, 2023 22:38
@sadmuphin
Copy link
Contributor Author

@richbeales Would it be an idea to include a flag that would create an example ai_settings.yaml for user reference? Or include an examples folder where we can store example files?

@richbeales richbeales merged commit 100fd8d into Significant-Gravitas:master Apr 14, 2023
SquareandCompass pushed a commit to SquareandCompass/Auto-GPT that referenced this pull request Oct 21, 2023
* mathchat blog

* Update website/blog/2023-06-28-MathChat/index.mdx

Co-authored-by: Chi Wang <[email protected]>

* Update website/blog/2023-06-28-MathChat/index.mdx

Co-authored-by: Chi Wang <[email protected]>

* Update website/blog/2023-06-28-MathChat/index.mdx

Co-authored-by: Chi Wang <[email protected]>

* Update website/blog/2023-06-28-MathChat/index.mdx

Co-authored-by: Qingyun Wu <[email protected]>

* Update website/blog/2023-06-28-MathChat/index.mdx

Co-authored-by: Qingyun Wu <[email protected]>

* Update website/blog/2023-06-28-MathChat/index.mdx

Co-authored-by: Qingyun Wu <[email protected]>

* update

* forrmat

* update

* delete image

* change link

---------

Co-authored-by: Chi Wang <[email protected]>
Co-authored-by: Qingyun Wu <[email protected]>
sindlinger pushed a commit to Orgsindlinger/Auto-GPT-WebUI that referenced this pull request Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants